Drop root windows from some internal apis

_gdk_device_query_state was needlessly shuffling
root windows around.

Drop keynav for dnd

This is very much an easter egg, and it is the last place where
we use the root window in libgtk.

adwaita: remove min-<lenght> from levelbar blocks

It's not needed and causes issues, see for details:
https://bugzilla.gnome.org/show_bug.cgi?id=783649

adwaita: Remove menu margins

menu margins have been added has a hack to mitigate bug:
https://bugzilla.gnome.org/show_bug.cgi?id=591258 with gtk+4 this
doesn't work anymore on gtk+4, the margin should probably be moved
to the parent window node, but it's not selectable, commenting out
for now.

Update Polish translation

widget: Remove gtk_widget_set_redraw_on_alloc

Since gtk+ draws more than the widget and allocates more size to it than
it knows about, this flag doesn't work anymore. Removing it (or setting
it to TRUE for widgets that used to set it to FALSE) fixes drawing
invalidation when these widgets get allocated a new size.

GdkWindow: Remove gdk_window_new_input

Not needed anymore.

Duplicate the TIC strings before NULLing the GObject data.

NULLing the data strings in GObject frees and invalidates the pointers
returned by g_object_get_data so we need our own copy of the values.

Revert "levelbar: Don't underallocate block widgets"

This reverts commit e25e1c54a44264de628fb3919fb47afd44e21ba5.

That commit causes a block always to be shown at the min side, even when
the value is 0, in which case, no fill should be shown.

https://bugzilla.gnome.org/show_bug.cgi?id=783649

3.91.2

Fix docs build

Container: Don’t scroll to unset focus child coord

In gtk_container_real_set_focus_child(), we try to scroll to the
position of the new :focus-child if we have h or v adjustments.

gtk_widget_translate_coordinates() returns FALSE if neither widget is
realized or in other situations that cause output parameters x and y not
to be set. Thus, if the caller did not initialise x/y and uses them even
if the function returned FALSE, they are using uninitialised variables.

In gtk_container_real_set_focus_child(), we did not check the return
value but merrily went ahead and used x and y regardless. This is UB, as
revealed by Valgrind, as well as being pointless.

The trivial fix is to exit early if (!gtk_widget_translate_coordinates).

https://bugzilla.gnome.org/show_bug.cgi?id=776909

Container: Fix scrolled coord in set_focus_child()

Commit 885bcd9fe4b6b4ecb003570ea0520cf42ec737a9 trampled the bit here
that is meant to translate between the nominated focus child and the
actual innermost one that is used for updating the h/v adjustments.

So, we need to save the passed focus child before diving into its
children, then translate and get allocations between them both. This
makes GTK+ 4 behave like GTK+ 3 again: instead of priv->focus_child and
focus_child, we now have focus_child and child, serving the roles of the
nominated focus child and its innermost focus child respectively.

This also ditches the unnecessary call to Widget:get_focus_child(), as
Container::set_focus_child() gets that same new child as an argument.

AccelLabel: set_accel_closure() arg is nullable

AccelLabel: set_accel_widget() arg is nullable

Updated Slovenian translation

Updated Slovenian translation

Update Friulian translation

window: Fix traversal of popovers during picking

These need to be traversed in the opposite direction. Given the order is
opposite to rendering, use a GQueue so it's cheap to traverse/add either
way.

HeaderBar: Use less clashing appmenu fallback icon

process-stop-symbolic is unintuitive if represented as a stop sign as in
Adwaita, and completely ambiguous if represented as a cross like the
window close button in other icon themes.

Instead, use application-x-executable, which is already used elsewhere
as a fallback if no specific icon can be found for the application.

https://bugzilla.gnome.org/show_bug.cgi?id=784624

docs: gtk/running: Document the GTK_CSD env var

It was not documented before.

https://bugzilla.gnome.org/show_bug.cgi?id=778791#c1

AccelLabel: Fix typos of Ctl to Ctrl

icon-browser: Add missing document-edit icon

https://bugzilla.gnome.org/show_bug.cgi?id=777515

gdkdisplay: Remove a pointless assignment

https://bugzilla.gnome.org/show_bug.cgi?id=784016

Enough with the beeping!

Don't beep when modifiers are released in entries.
This was an inadvertent change that snuck in with
the emoji support.

Also, don't beep while entering an emoji name.
There is entirely too much beeping here.

Use the right gettext domain

Most of our .ui files were still using the gtk30 domain,
despite the library having switched to gtk40. Fix this.

AccelLabel: Fix displaying accel unichars >= 0x80

In GTK+ 2, the ch < 0x80 was ORd with klass->latin1_to_char, and that
was unconditionally set to TRUE in the class init function, so
effectively the ch < 0x80 never mattered before or served any purpose.

When klass->latin1_to_char was deleted from the class in commit
f760538f17673c5bd7fec792be2f1abf8148fc32, this check’s sense changed.

The resuls was that accel keyvals with gunichar value >= 0x80 stopped
being rendered as symbols, instead falling back to their keysym name.
Instead of recognisable symbols for these, we get raw, often obscure,
and untranslatable keysym names. This breaks accessibility as well as
client users who may be parsing such accels and migrating from GTK+ 2.

So, remove the < 0x80 to restore the behaviour from before said commit.

https://bugzilla.gnome.org/show_bug.cgi?id=783906

CenterBox: minor tweaks to doc blurb

CenterBox: Add missing nullable annotations

https://bugzilla.gnome.org/show_bug.cgi?id=785280

CenterBox: Guard against directly including header

https://bugzilla.gnome.org/show_bug.cgi?id=785280

centerbox: Add G_BEGIN_DECLS and G_END_DECLS

https://bugzilla.gnome.org/show_bug.cgi?id=785280

Add support for entering emoji by name

This commit adds some basic support for entering emoji by name
to GtkIMContextSimple. To begin an emoji sequence, use Ctrl-Shift-e
instead of Ctrl-Shift-u that is used for hex input. Otherwise, the
behavior is the same: you can can let go of the modifier keys and
end the sequence with space or enter, or hold on to the modifier
keys and end the sequence by releasing them.

Only a limited, fixed set of names is supported at this time, see
the GtkIMContextSimple docs for a full list.

gskrendernodeimpl: Fix leak of cairo_pattern_t

It was not destroyed after setting it on the cairo_context_t, as it is
in every other case.

https://bugzilla.gnome.org/show_bug.cgi?id=785793

Window: Also quote Window address in Popover error

This is symmetrical with how we already do that for the Popover, and it
could be useful for debugging.

widget-factory: Add tooltip to Entry icon

just to show we can.

Fix some nullable Returns annotations

based on reports and patches by Iñaki García Etxebarria:

https://bugzilla.gnome.org/show_bug.cgi?id=781246
https://bugzilla.gnome.org/show_bug.cgi?id=785423

inspector: Fix object title label alignment

Re-center it after the center-child removal in GtkBox and the addition
of GtkCenterBox.

GtkFixed: Update documentation intro

• Add GtkLayout as a @See_also since it includes fixed-pos functionality
• Drop mention of the long-gone Linux framebuffer port
• Explain how to work around the problems with RTL text

Updated Czech translation

Layout: Tweak documentation blurb

Being addable to a ScrolledWindow is not interesting; now that SW
auto-adds a Viewport if needed, so can DrawingArea and any other widget.

Mention GtkFixed in case the reader just wants that bit of functionality

entry: get_icon_area returns in entry coordinates

This fixes the popover positions in the gtk4-demo/popovers, and it also
just makes sense. Also document what the returned coordinates are
relatiev to.

popover: Fix arrow position

Remove all the manual css margin handling since that's being done for us
automatically now.

Update POTFILES.in

eventbox: Remove

texthandle: Don't use GtkEventBox

testsuite: Don't use GtkEventBox in gestures test

a11y: Remove check for GtkEventBox

docs: Don't use GtkEventBox

testsuite: Remove GtkEventBox from a11y test

iconbrowser: Remove GtkEventBox usage

gtk-demo: Get rid of GtkEventBox

cellaccel: Get rid of GtkEventBox

Can't test the code atm because treeview editing is broken, let's hope
it works once that's fixed.

placessidebar: Remove EventBox usage

placesview: Remove event box usage

meson: Add missing Wayland protocol

GDK depends on a new Wayland protocol.

wayland: add shortcut inhibitor support

This adds support for the shortcut inhibitor protocol in gdk/wayland
backend.

A shortcut inhibitor request is issued from the gdk wayland backend for
both the older, deprecated API gdk_device_grab() and the new gdk seat
API gdk_seat_grab(), but only if the requested capability is for the
keyboard only.

https://bugzilla.gnome.org/show_bug.cgi?id=783343

reftests: Always wait for rendering to settle

We wait for a few 100ms for rendering to settle in various WMs. So far
we only did that for windows that were controlled by the WM (aka
toplevels).

With modern compositing gnome-shell however, this now also applies to
override-redirect windows, so we now wait there, too.

This makes the reftests a lot slower, but they now actually work when
running make check in gnome-shell.

reftests: Remove event box usage

testuite: Remove EventBox from builder test

Updated Slovenian translation

widget: Constify state parameter of propagate_state

Makes sense.

widget: Don't try to propagate 0 state flags

This happens quite often and has no effect anyway.

widget: Fix size_allocate's out_clip annotation

testgtk: Actually expand the scrolled window

testgtk: Get rid of GtkEventBox

tests: Get rid of GtkEventBox

bindings: Avoid iterating array if it is never set

If query.return_type is not one we want, binding_compose_params() is
not called, and so params remains a NULL pointer. However, the code was
then unconditionally iterating it regardless. Don't if it is still NULL.

CID 1452218 (#1 of 1): Explicit null dereferenced (FORWARD_NULL)
15. var_deref_op: Dereferencing null pointer params.

cssselector: Ensure we do not index out of bounds

This would only happen if the last element was deprecated, but it should
be avoided anyway.

CID 1388852 (#1 of 1): Out-of-bounds read (OVERRUN)
12. overrun-local: Overrunning array pseudo_classes of 16 32-byte
elements at element index 16 (byte offset 512) using index i + 1U (which
evaluates to 16).

Notebook: Remove null check after dereference

This function clearly assumes the parameter children cannot be NULL, and
the call sites seem to perform enough checks to confirm this.

CID 1388869 (#1 of 1): Dereference before null check (REVERSE_INULL)
check_after_deref: Null-checking children suggests that it may be null,
but it has already been dereferenced on all paths leading to the check.

Calendar: Add default case to appease Coverity

CID 1432024 (#1 of 1): Uninitialized scalar variable (UNINIT)
2. uninit_use_in_call: Using uninitialized value rect.x when calling
calendar_arrow_rectangle.

Add a default case to the switch which will bail out with
g_assert_not_reached(), which should reassure Coverity that the method
is always called with a valid value that is handled in the switch.

cssbordervalue: Fix possible use of uninitialised

If value->values[i] is NULL, then values[i] was left uninitialised.
The code then reads each element of values[].

CID 1432029 (#1 of 1): Uninitialized pointer read (UNINIT)
11. uninit_use: Using uninitialized value values[i].

Stack: Fix copy-paste error

This fixes Coverity CID 1452217.

Entry: Fix unset icon tooltip hiding Entry tooltip

Our ::query-tooltip handler first checks whether the pointer is over any
of the icons, returning their tooltip if so, and if not chains up to
Widget::query-tooltip in order to show the text for the widget overall.

But ensure_has_tooltip(), which exists to update :has-tooltip based on
whether ::query-tooltip is needed, only set :has-tooltip to TRUE if any
icon had a tooltip, without caring whether the widget as a whole does.

That is asymmetrical and meant that if the Entry had a tooltip, but
subsequently all icons had their tooltips unset, :has-tooltip would be
set to FALSE, and hence the tooltip for the widget would become lost.

The fix is to set :has-tooltip to TRUE if the widget has a tooltip of
its own, and we only need to check the icons if that is not the case.

https://bugzilla.gnome.org/show_bug.cgi?id=785672

Entry: Warn about corner case hiding icon tooltips

https://bugzilla.gnome.org/show_bug.cgi?id=785672#c4

widget: Unset PRELIGHT and ACTIVE state in _set_sensitive

They both depend on input, which an insensitive widget does not receive.

button: Stop fiddling with PRELIGHT state

This is done automatically now and it's wrong here since it gets
executed after gesture release.

scale: Allocate the value next to the slider

... when vertical. Also fixes usage of the uninitialized value_alloc.y.

Visual Studio builds: Adapt to new glib-mkenums

glib-mkenums is now done in Python, but since the Visual Studio build
environment (cmd.exe) does not support shebang lines, we need to call
the interpretor explicitly to run the script.

This means that we need to update on how we generate
gsk/gskenumtypes.[c|h] in our projects, as at this point GTK+-3.91.x
does not require a GLib installation that ships with the Python-fied
glib-mkenums.  As a result, we adapt to this by first using Python
to call glib-mkenums.  If this fails (where the output file becomes 0
in size), then we use PERL to call the glib-mkenums script.  Note that
during the build this will cause a warning message to be displayed,
stating that '&' cannot be found, but due to the way Windows .bat script
are done, we need to live with that until a solution can be found on
this.

This is likely a problem that does not exist in the Meson builds, as
Meson will take care of calling the interpretor for us by looking at
the shebang lines for our case.

Also, clean up the .batin Windows batch script that is used to call
glib-mkenums by using a for loop in there.

testentryicons: Set tooltips on all icons

Just to test tooltips in all cases; what was already here
should have been sufficient, but this doesn't hurt.

While here, also add some instructive placeholder text.

https://bugzilla.gnome.org/show_bug.cgi?id=780938

MSVC builds: Allow different Python versions per toolset

Use conditionals to select the Python installation, so that we can stick
closer to the default Visual Studio versions used to compile each official
Python releases.

This means by default:
-2013 builds use Python 3.4.x, which is built with 2010
-2015 and 2017 builds use Python 3.6.x, which is built with 2015.

Also rename PythonPath/PythonPathX64 in the property sheets to
PythonDir/PythonDirX64 repsectively, as PythonPath is the envvar name
where additional Python modules is searched for, so we don't want to get
confused with it.

Last but not least, distinguish between the Python interpretors that are
used on x64 and x86/32-bit builds for generating the libgtk4.manifest
file and the gdbus-generated sources, for consistency reasons.

entry: Remove unused/duplicate widget in IconInfo

widget: Don't access highlight_resize in non-debug builds

The GtkWidgetPrivate struct only contains that member if G_ENABLE_DEBUG
is defined.

eventbox: Remove visible-window property

The GdkWindow that was supposed to back it is gone.

Updated Slovenian translation

Updated Slovenian translation

Updated Slovenian translation

gtklabel: Fix touch link handling under wayland

Refactor the code updating the active link under the current coordinates
into a separate function, and call it on GtkGestureMultiPress::pressed
so the link is updated on GDK_TOUCH_BEGIN. Based on a patch by
Jan-Michael Brummer <jan.brummer@tabos.org>.

https://bugzilla.gnome.org/show_bug.cgi?id=776903

gtklinkbutton: Set cursor on button's event window

It is not necessary to (re)set the cursor on every crossing
event, and can probably yield the wrong results if there are
multiple master devices involved. Just set it on init(), and
let the inner machinery update the cursor whenever necessary.

This patch is an adaption of commit 0daf79676 in gtk-3-22, the
side effects are not as bad here because the cursor was already
being set on the widget specifically instead of the parent
widget's, but there's still some nonetheless (plus, it's simpler)

https://bugzilla.gnome.org/show_bug.cgi?id=785375

wayland: Clear tablet tool cursor on proximity out

This ensures that the tablet tool will get the cursor updated even if it
happens to fall within the same window again on the next proximity in.

https://bugzilla.gnome.org/show_bug.cgi?id=785375

gdkwindow: Update cursor for tablet tools in the window

This check must be done explicitly on Wayland as the master device for
tablet tools differ from the Core Pointer. This ensures that whenever a
tablet tool is inside a window and the cursor is programmatically changed,
it will be visually updated too.

https://bugzilla.gnome.org/show_bug.cgi?id=785375

wayland: Observe GDK_SEAT_CAPABILITY_TABLET_STYLUS on gdk_seat_get_slaves()

This flag wasn't being honored so far...

Updated Spanish translation

Update Croatian translation

actionbar: Call g_object_class_install_properties()

https://bugzilla.gnome.org/show_bug.cgi?id=785318

Avoid the use of g_autofree

We can't use this in code that may be built on win32,
so lets just not use it at all.

widget: Stop invalidating recursively

updated Punjabi Translation

Update Friulian translation

notebook: Set tabs child visible before allocating a size

Otherwise we might skip them here, then set child-visible to TRUE and
then later snapshot them even though they have no valid size.

cssstylechange: Fix change printing

Only print the old value once, above the new one.